Skip to content

Conversation

@vicb
Copy link

@vicb vicb commented Feb 3, 2014

Looks a bit hacky for now as -y --force-yes do not seem to work but it'll do the trick while 2.4 is not available by default

vicb added 3 commits February 3, 2014 10:16
- Use the latest avaiable HHVM
- Update composer
- composer --dev is the default
@vicb vicb mentioned this pull request Feb 3, 2014
@lsmith77
Copy link
Member

lsmith77 commented Feb 3, 2014

ah ok .. so the issue is fixed in 2.4 already? afaik there is already a cookbook for hhvm 2.4 for travis, so its just a question until it gets deployed.

@vicb
Copy link
Author

vicb commented Feb 3, 2014

@lsmith77 right, I put some details in the original PR.

@dbu
Copy link
Member

dbu commented Feb 3, 2014

thanks @vicb for the PR.

yep, its fixed there. we are not in a huge hurry for hhvm, i would prefer to wait for travis to update their system over adding this here. the big blocker is the missing libxml support in hhvm, before that is done, jackalope won't run on hhvm anyways.

@lsmith77
Copy link
Member

lsmith77 commented Feb 3, 2014

@vicb thx for investigating this for us!

@dbu
Copy link
Member

dbu commented Feb 5, 2014

thanks again @vicb - i will close this and wait for travis to update their setup. their PR was merged already so i guess its going to be soon.

@dbu dbu closed this Feb 5, 2014
@vicb
Copy link
Author

vicb commented Feb 5, 2014

no pb. Hopefully libxml support will be integrated soon.

@dbu
Copy link
Member

dbu commented Feb 5, 2014

yeah, that would be great. seems like facebook themselves do not need
it, unfortunately

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants